Skip to content

Conversation

tordtst
Copy link
Member

@tordtst tordtst commented Sep 29, 2026

Endringer

  • Prettier med enkle anførselstegn, semikolon, 2 mellomrom og trailingComma: all (.prettierrc.json).
  • .prettierignore holder dist, node_modules, coverage og package-lock.json utenfor formateringen.
  • ESLint 10 (flat config) med typescript-eslint, React Hooks-regler og react-refresh for Vite.
  • eslint-config-prettier ligger sist i eslint.config.js, så ESLint ikke overstyrer Prettier.
  • Nye kommandoer: npm run lint, npm run format og npm run format:check.
  • README dokumenterer kommandoene og hvordan ESLint og Prettier deler ansvaret.

Hvorfor

Gi gruppen samme kodestil og oppdag enkle feil tidlig, før resten av prosjektet bygges.

Issue

Closes #2

tordtst and others added 3 commits September 29, 2026 17:20
Gi gruppen lik formatering med enkle anførselstegn, semikolon, to
mellomrom og trailingComma: all. Byggfiler, avhengigheter og lockfil
holdes utenfor formateringen.

Refs #2

Co-Authored-By: birkjr <birkjr@stud.ntnu.no>
Oppdag vanlige feil som betingede hooks, any og ubrukte variabler
tidlig. eslint-config-prettier ligger sist slik at ESLint ikke
overstyrer formateringen fra Prettier.

Refs #2

Co-Authored-By: sebasti <sebasti@stud.ntnu.no>
Vis hvordan lint og formatering kjøres, og hvordan ESLint og
Prettier deler ansvaret.

Refs #2

Co-Authored-By: andretg <andretg@stud.ntnu.no>
@tordtst tordtst added this to the Innlevering 1 milestone Sep 29, 2026
@tordtst tordtst self-assigned this Sep 29, 2026
@tordtst tordtst requested a review from a team as a code owner September 29, 2026 15:22
@tordtst tordtst linked an issue Sep 29, 2026 that may be closed by this pull request
5 tasks
@tordtst tordtst changed the base branch from main to dev September 29, 2026 15:23
Copy link
Member

@andretg andretg left a comment •

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testet lokalt: npm ci, lint, typecheck, build og dev fungerer. Oppsettet er dokumentert i README.
Merk: npm run format:check feiler for alle filer hos meg, men dette kan vi fikse i en annen issue med npm run format.

Godkjenner denne PR-en.

@andretg andretg merged commit 633e393 into dev Sep 29, 2026
@andretg andretg deleted the 2-config-sett-opp-linting-og-formatering branch September 29, 2026 15:42
Sign in to join this conversation on GitHub.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CONFIG] Sett opp linting og formatering
2 participants